Skip to content

[Docs] Add applies_to to DNS processor docs#45580

Merged
vishaangelova merged 3 commits intomainfrom
add-applies-to-tags-dns
Jul 28, 2025
Merged

[Docs] Add applies_to to DNS processor docs#45580
vishaangelova merged 3 commits intomainfrom
add-applies-to-tags-dns

Conversation

@vishaangelova
Copy link
Copy Markdown
Member

@vishaangelova vishaangelova commented Jul 28, 2025

Proposed commit message

Adds applies_to tags to changes in the documentation introduced with #44932

Reason for change

Note

Starting with v9.0, there is no longer a new documentation set published with every minor release: the same page stays valid over time and shows version-related evolutions. Read more in Write cumulative documentation.

In #44932, @mjmbischoff added the option to disable the DNS processor failure or success cache. Based on #45077 (comment), the feature will be available in v9.1+. We should add 9.1 applies_to labels to the processor-dns.md docs for the different beats.

@vishaangelova vishaangelova requested a review from a team as a code owner July 28, 2025 14:24
@botelastic botelastic Bot added the needs_team Indicates that the issue/PR needs a Team:* label label Jul 28, 2025
@github-actions
Copy link
Copy Markdown
Contributor

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@vishaangelova vishaangelova added the Team:Docs Label for the Observability docs team label Jul 28, 2025
@botelastic botelastic Bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Jul 28, 2025
@vishaangelova vishaangelova added the backport-9.1 Automated backport to the 9.1 branch label Jul 28, 2025
@mergify
Copy link
Copy Markdown
Contributor

mergify Bot commented Jul 28, 2025

This pull request does not have a backport label.
If this is a bug or security fix, could you label this PR @vishaangelova? 🙏.
For such, you'll need to label your PR with:

  • The upcoming major version of the Elastic Stack
  • The upcoming minor version of the Elastic Stack (if you're not pushing a breaking change)

To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-8./d is the label to automatically backport to the 8./d branch. /d is the digit
  • backport-active-all is the label that automatically backports to all active branches.
  • backport-active-8 is the label that automatically backports to all active minor branches for the 8 major.
  • backport-active-9 is the label that automatically backports to all active minor branches for the 9 major.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jul 28, 2025

@vishaangelova vishaangelova changed the title Add applies_to to DNS processor docs [Docs] Add applies_to to DNS processor docs Jul 28, 2025
Copy link
Copy Markdown
Contributor

@colleenmcginnis colleenmcginnis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The applies_to badges look great! I left some suggestions below to fix the format of the definition list items, which was broken before this PR. A definition list in Markdown is created by writing a term on one line, followed by a colon and at least three spaces (doc).

Comment thread docs/reference/auditbeat/processor-dns.md Outdated
Comment thread docs/reference/auditbeat/processor-dns.md Outdated
Comment thread docs/reference/filebeat/processor-dns.md Outdated
Comment thread docs/reference/filebeat/processor-dns.md Outdated
Comment thread docs/reference/heartbeat/processor-dns.md Outdated
Comment thread docs/reference/metricbeat/processor-dns.md Outdated
Comment thread docs/reference/packetbeat/processor-dns.md Outdated
Comment thread docs/reference/packetbeat/processor-dns.md Outdated
Comment thread docs/reference/winlogbeat/processor-dns.md Outdated
Comment thread docs/reference/winlogbeat/processor-dns.md Outdated
Co-authored-by: Colleen McGinnis <colleen.mcginnis@elastic.co>
Copy link
Copy Markdown
Contributor

@colleenmcginnis colleenmcginnis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vishaangelova vishaangelova merged commit 735bd17 into main Jul 28, 2025
14 checks passed
@vishaangelova vishaangelova deleted the add-applies-to-tags-dns branch July 28, 2025 16:49
mergify Bot pushed a commit that referenced this pull request Jul 28, 2025
* Add applies_to to DNS processor docs

* Apply suggestions from code review

Co-authored-by: Colleen McGinnis <colleen.mcginnis@elastic.co>

---------

Co-authored-by: Colleen McGinnis <colleen.mcginnis@elastic.co>
(cherry picked from commit 735bd17)
vishaangelova added a commit that referenced this pull request Jul 28, 2025
* Add applies_to to DNS processor docs

* Apply suggestions from code review



---------


(cherry picked from commit 735bd17)

Co-authored-by: Visha Angelova <91186315+vishaangelova@users.noreply.github.com>
Co-authored-by: Colleen McGinnis <colleen.mcginnis@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-9.1 Automated backport to the 9.1 branch Team:Docs Label for the Observability docs team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants